From: Cosimo Cecchi <cosimoc@gnome.org>
Date: Tue, 19 Jun 2012 19:17:07 +0000 (-0400)
Subject: autorun: simplify code
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~6453
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=ebee01b3550d102d722aaba760d1482fe9bcb8ee;p=gnome-shell.git

autorun: simplify code

Split autorun conditions into separate logic blocks instead of a single
huge if.

https://bugzilla.gnome.org/show_bug.cgi?id=660595
---

diff --git a/js/ui/autorunManager.js b/js/ui/autorunManager.js
index 9d39690e9..7283e16cd 100644
--- a/js/ui/autorunManager.js
+++ b/js/ui/autorunManager.js
@@ -23,12 +23,14 @@ const AutorunSetting = {
 };
 
 // misc utils
-function ignoreAutorunForMount(mount) {
+function shouldAutorunMount(mount) {
     let root = mount.get_root();
     let volume = mount.get_volume();
 
-    if ((root.is_native() && !isMountRootHidden(root)) &&
-        (volume && volume.allowAutorun && volume.should_automount()))
+    if (!volume || !volume.should_automount() || !volume.allowAutorun)
+        return false;
+
+    if (!root.is_native() || isMountRootHidden(root))
         return false;
 
     return true;
@@ -270,7 +272,7 @@ const AutorunResidentSource = new Lang.Class({
     },
 
     addMount: function(mount, apps) {
-        if (ignoreAutorunForMount(mount))
+        if (!shouldAutorunMount(mount))
             return;
 
         let filtered = this._mounts.filter(function (element) {
@@ -448,7 +450,7 @@ const AutorunTransientDispatcher = new Lang.Class({
             return;
 
         // if the mount doesn't want to be autorun, return
-        if (ignoreAutorunForMount(mount))
+        if (!shouldAutorunMount(mount))
             return;
 
         let setting = this._getAutorunSettingForType(contentTypes[0]);